Skip to content

Add welcome modal and guide entry#129

Open
Jiangzhewen wants to merge 1 commit into
capofficial:mainfrom
Jiangzhewen:welcome-modal-bridge
Open

Add welcome modal and guide entry#129
Jiangzhewen wants to merge 1 commit into
capofficial:mainfrom
Jiangzhewen:welcome-modal-bridge

Conversation

@Jiangzhewen
Copy link
Copy Markdown

Closes #13

Summary

  • Add a first-visit welcome modal for dashboard users after the country check completes.
  • Persist dismissal in existing user settings so experienced users are not interrupted again.
  • Add a Guide nav entry on desktop and mobile to reopen the onboarding modal at any time.
  • Include Trade, Pool, Stake, Arbitrum Bridge, and wallet setup entry points without adding wallet signing or bridge SDK code.

Validation

  • npm ci
  • npm run build
  • git diff --check
  • Browser validation with api.country.is mocked to an allowed country: first visit shows the modal, Got it dismisses it, reload does not show it again, desktop Guide reopens it, and mobile menu Guide reopens it.

@Jiangzhewen
Copy link
Copy Markdown
Author

Local validation completed:

  • npm ci
  • npm run build
  • git diff --check
  • Browser validation with api.country.is mocked to an allowed country: first visit shows the modal, Got it dismisses it, reload does not show it again, desktop Guide reopens it, and mobile menu Guide reopens it.

The Vercel status check currently fails with Vercel's “account deployment blocked” page, which appears to be a project/integration account issue rather than a branch build error.

@Jiangzhewen
Copy link
Copy Markdown
Author

Vercel is failing before it reaches the project build with Account is blocked and links to Vercel's account-blocked help page, so this appears to be an account/deployment permission issue rather than a code failure. I re-ran npm run build locally on the current PR head and it completes successfully; the output only contains the existing Rollup/Svelte warnings already present in this project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Welcome modal with integrated bridge and/or instructions

1 participant